Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use external GraphQL client for smoke tests #179

Merged
merged 9 commits into from
Aug 17, 2023

Conversation

leonardehrenfried
Copy link

Since GraphQL is now the main API that the IBI frontend use, I'm switching the SmokeTests to also use GraphQL.

It does this by using a separate client that I wrote for another project.

Copy link

@miles-grant-ibigroup miles-grant-ibigroup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonderful changes thanks

Copy link

@binh-dam-ibigroup binh-dam-ibigroup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


@Test
public void regularRouteFromCentralAtlantaToPowderSprings() {
Set<String> modes = Set.of("TRANSIT", "WALK");
var modes = Set.of(TRANSIT, WALK);
SmokeTest.basicRouteTest(
new SmokeTestRequest(nearGeorgiaStateStation, powderSpringsInsideFlexZone1, modes),
List.of("WALK", "SUBWAY", "WALK", "BUS", "WALK", "BUS", "WALK")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The expected modes could eventually be moved to use LegMode directly.

@leonardehrenfried leonardehrenfried merged commit f0df87c into ibi-dev-2.x Aug 17, 2023
14 checks passed
@leonardehrenfried leonardehrenfried deleted the graphql-smoke-tests branch August 17, 2023 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants